Skip to content

fix: Select component highlighted text color#7196

Merged
bmuenzenmeyer merged 1 commit intonodejs:mainfrom
canerakdas:fix/select-highlighted-text-color
Nov 7, 2024
Merged

fix: Select component highlighted text color#7196
bmuenzenmeyer merged 1 commit intonodejs:mainfrom
canerakdas:fix/select-highlighted-text-color

Conversation

@canerakdas
Copy link
Copy Markdown
Member

Description

With the update made in the PR #7179, the theme strategy was changed from class to selector.

This gives the styling given to the attribute more priority than the styling given to the class. With this PR, a highlighted text color for dark mode was added to the Select component

Validation

In the preview build highlighted text should be visible

Related Issues

fixes #7185

Check List

  • I have read the Contributing Guidelines and made commit messages that follow the guideline.
  • I have run npm run format to ensure the code follows the style guide.
  • I have run npm run test to check if all tests are passing.
  • I have run npx turbo build to check if the website builds without errors.
  • I've covered new added functionality with unit tests if necessary.

@canerakdas canerakdas added the bug label Nov 7, 2024
@canerakdas canerakdas requested a review from a team as a code owner November 7, 2024 11:32
@vercel
Copy link
Copy Markdown

vercel Bot commented Nov 7, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
nodejs-org ✅ Ready (Inspect) Visit Preview Nov 7, 2024 11:34am

Copy link
Copy Markdown
Contributor

@bmuenzenmeyer bmuenzenmeyer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM - thanks

@bmuenzenmeyer bmuenzenmeyer added the fast-track Fast Tracking PRs label Nov 7, 2024
@bmuenzenmeyer
Copy link
Copy Markdown
Contributor

fast tracking this - this is broken in production

Copy link
Copy Markdown
Member

@AugustinMauroy AugustinMauroy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGMT !

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Nov 7, 2024

Unit Test Coverage Report

Lines Statements Branches Functions
Coverage: 92%
90.5% (591/653) 76.06% (178/234) 94.35% (117/124)

Unit Test Report

Tests Skipped Failures Errors Time
132 0 💤 0 ❌ 0 🔥 5.115s ⏱️

@bmuenzenmeyer bmuenzenmeyer added this pull request to the merge queue Nov 7, 2024
Merged via the queue into nodejs:main with commit edfa6fc Nov 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fast-track Fast Tracking PRs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

UI/UX: when i hover over the dropdown the list item that i am hovering over gets disappeared

3 participants